Skip to content

feat: use WaitJobRaw for process_tool#635

Merged
GabrielVasilescu04 merged 1 commit intomainfrom
feature/use-job-raw-trigger
Mar 2, 2026
Merged

feat: use WaitJobRaw for process_tool#635
GabrielVasilescu04 merged 1 commit intomainfrom
feature/use-job-raw-trigger

Conversation

@GabrielVasilescu04
Copy link
Contributor

@GabrielVasilescu04 GabrielVasilescu04 commented Feb 26, 2026

Description

Currently, if the job started by the process_tool fails, then the agent execution fails as a whole. Replace the WaitJob trigger with WaitJobRaw which should prevent this from happening. If the job's state is faulted, WaitJob raises an error, but WaitJobRaw simply returns the job object.
New trigger type was addded in this PR: https://github.com/UiPath/uipath-platform-python/pull/8

Development Package

  • Use uipath pack --nolock to get the latest dev build from this PR (requires version range).
  • Add this package as a dependency in your pyproject.toml:
[project]
dependencies = [
  # Exact version:
  "uipath-langchain==0.7.15.dev1006353189",

  # Any version from PR
  "uipath-langchain>=0.7.15.dev1006350000,<0.7.15.dev1006360000"
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath-langchain = { index = "testpypi" }

[tool.uv]
override-dependencies = [
    "uipath-langchain>=0.7.15.dev1006350000,<0.7.15.dev1006360000",
]

@GabrielVasilescu04 GabrielVasilescu04 self-assigned this Feb 26, 2026
@GabrielVasilescu04 GabrielVasilescu04 force-pushed the feature/use-job-raw-trigger branch 2 times, most recently from 553122c to 4ef24e3 Compare February 26, 2026 15:24
@GabrielVasilescu04 GabrielVasilescu04 force-pushed the feature/use-job-raw-trigger branch from 4ef24e3 to 0e71565 Compare February 26, 2026 15:51
@GabrielVasilescu04 GabrielVasilescu04 marked this pull request as ready for review February 26, 2026 15:56
Comment on lines +84 to +85
error_info = str(job.job_error or job.info or "Unknown error")
return f"Process did not finish successfully. Error: {error_info}"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's not add additional Process did not finish successfully. Error: as we only had job.info handling in the previous execution engine and would like to convey complete parity atm

return await start_job()
job = await start_job()

if (job.state or "").lower() == "faulted":
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could we use an enum exposed by the orchestrator_service with the job_state?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

@cotovanu-cristian cotovanu-cristian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good after we fix the status exported from uip-platform

@GabrielVasilescu04 GabrielVasilescu04 force-pushed the feature/use-job-raw-trigger branch 2 times, most recently from 488792f to b29b4dc Compare March 2, 2026 11:12
@GabrielVasilescu04 GabrielVasilescu04 force-pushed the feature/use-job-raw-trigger branch from b29b4dc to b7abb1c Compare March 2, 2026 11:15
@GabrielVasilescu04 GabrielVasilescu04 merged commit a5319af into main Mar 2, 2026
39 checks passed
@GabrielVasilescu04 GabrielVasilescu04 deleted the feature/use-job-raw-trigger branch March 2, 2026 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants